Skip to content

make selection machinery pluggable so that for instance we can use an… - #1537

Merged
reiern70 merged 7 commits into
wicketstuff:masterfrom
reiern70:reiern70/custom-selection
Sep 3, 2026
Merged

reiern70 merged 7 commits into
wicketstuff:masterfrom
reiern70:reiern70/custom-selection

Conversation

@reiern70

Copy link
Copy Markdown
Contributor

… ID based mechanism

@solomax solomax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need tests for this functionality? :)

Comment thread pom.xml
<link>https://nightlies.apache.org/wicket/apidocs/11.x</link>
<link>https://docs.spring.io/spring-framework/docs/6.0.x/javadoc-api</link>
<link>https://logback.qos.ch/apidocs</link>
<link>https://javadoc.io/doc/ch.qos.logback/logback-classic/1.5.18</link>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this one :))

@reiern70
reiern70 requested a review from solomax September 2, 2026 19:57
@reiern70

reiern70 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

Do we need tests for this functionality? :)

I will see what I can do. I tested examples and they are still working. I can also add some exaple no how to use this feature.

@reiern70

reiern70 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

Do we need tests for this functionality? :)

I will see what I can do. I tested examples and they are still working. I can also add some exaple no how to use this feature.

It seems there are no tests regarding autocomplete. I just added some example

Screencast.From.2026-09-02.15-58-45.webm

@reiern70

reiern70 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

@solomax I included your feedback + some examples. I would appreciate another review

@reiern70
reiern70 merged commit 23823e8 into wicketstuff:master Sep 3, 2026
1 check passed
@reiern70
reiern70 deleted the reiern70/custom-selection branch September 3, 2026 13:00

This branch was previously deployed

1 inactive deployment
CI_DEPLOY — c6cc316f Deployed Sep 2, 2026 by reiern70 via Java 17 Test #1689
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants